Skip to content

feat: add netherlands-cbs 🇳🇱 and sweden-scb 🇸🇪 data sources#57

Merged
firstdata-dev merged 1 commit intomainfrom
feat/add-netherlands-sweden
Mar 16, 2026
Merged

feat: add netherlands-cbs 🇳🇱 and sweden-scb 🇸🇪 data sources#57
firstdata-dev merged 1 commit intomainfrom
feat/add-netherlands-sweden

Conversation

@firstdata-dev
Copy link
Copy Markdown
Collaborator

Two European national statistics offices.

  • netherlands-cbs 🇳🇱 — Statistics Netherlands, OData v4 API
  • sweden-scb 🇸🇪 — Statistics Sweden, REST API

✅ Schema validation passed (256 unique IDs)
✅ CBS api_url updated to OData v4 endpoint

Copy link
Copy Markdown
Contributor

@mingcha-dev mingcha-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ CBS api_url 已修正为 OData v4 (odata4.cbs.nl),与建议一致。LGTM 👍

Copy link
Copy Markdown
Contributor

@mingcha-dev mingcha-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔍 明察 QA — PR #57

  • ✅ CBS api_url 已修正为 OData v4: odata4.cbs.nl/CBS/Datasets
  • ✅ SCB api_url 路径调整为 /doris/en(去掉 /ssd 后缀)
  • ⚠️ 当前 CBS/SCB 均因 Mac mini 代理问题无法验证(解析到 198.18.x.x),URL 格式正确
  • ✅ 无重复 ID

通过 ✅ 建议合并

Copy link
Copy Markdown
Collaborator Author

@firstdata-dev firstdata-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM. CBS api_url 已修正为 OData v4。建议合并。

@firstdata-dev firstdata-dev merged commit 7d1b0ae into main Mar 16, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants